Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: merge test jobs #367

Merged
merged 10 commits into from
Dec 2, 2021
Merged

ci: merge test jobs #367

merged 10 commits into from
Dec 2, 2021

Conversation

redeboer
Copy link
Member

@redeboer redeboer commented Dec 2, 2021

Reorganise GitHub Action workflow for pytest.
Extracted from #366 in order to identify potential performance issues. Notebooks slow after the changes introduced in #366, which may be caused by the changes to the callbacks.

Other changes;

  • Treat warnings raised under pytest as errors.
  • Renamed tox job for testing notebooks to nb.

@redeboer redeboer added the 🖱️ DX Improvements to the Developer Experience label Dec 2, 2021
@redeboer redeboer added this to the 0.4.0 milestone Dec 2, 2021
@redeboer redeboer self-assigned this Dec 2, 2021
@redeboer redeboer enabled auto-merge (squash) December 2, 2021 17:38
@redeboer redeboer merged commit 4f9e330 into main Dec 2, 2021
@redeboer redeboer deleted the merge-tests branch December 2, 2021 17:46
redeboer added a commit that referenced this pull request Dec 7, 2021
* ci: deactivate fast-fail in Ubuntu tests
* ci: ignore RuntimeWarning numpy ufunc in Python 3.6
* ci: merge MacOS test job with Ubuntu test jobs
* ci: rename tox job testnb to nb
* ci: update pip constraints and pre-commit config
* ci: update pip constraints and pre-commit config
* ci: update workflow configs with repo-maintenance
* style: use string for python versions in GitHub workflow config

Co-authored-by: GitHub <noreply@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖱️ DX Improvements to the Developer Experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants